Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Log Form] Log templates and save last log text. #2451

Merged
merged 3 commits into from
Nov 18, 2023

Conversation

2Abendsegler
Copy link
Owner

@2Abendsegler 2Abendsegler self-assigned this Nov 16, 2023
@2Abendsegler 2Abendsegler added this to the v0.14.7 milestone Nov 16, 2023
@2Abendsegler 2Abendsegler changed the title [Log Form] Log Templates. [Log Form] Log templates and save last log text. Nov 17, 2023
@2Abendsegler 2Abendsegler marked this pull request as ready for review November 17, 2023 12:07
@2Abendsegler
Copy link
Owner Author

@capoaira
Moin,

die Features "Log templates" und "save last log text" zum Redesign sind fertig.
Kannst du bitte testen? Danke. 😊

Ich möchte gerne am Sonntag eine neue Version raus bringen, falls ich bis dahin noch die wichtigsten Punkte erledigt bekomme. Hast du Zeit zum Testen? Falls nicht, würde ich meine letzten PRs trotzdem mergen in der Hoffnung, dass ich nichts Wichtiges übersehen habe. Gib einfach Bescheid wenn du keine Zeit hast.

@2Abendsegler
Copy link
Owner Author

Habe noch zwei Punkte vergessen zu erwähnen:

  1. Das betrifft nicht nur das Thema Log Templates aus deiner Liste, sondern auch das Thema Replace log by last log template.
  2. Nach dem Einfügen eines Log Templates muss man noch eine Eingabe machen, damit der eingefügte Text tatsächlich vorhanden ist. Das ist das Problem aus Issue [New log form] Smiley inserted on the log page don't appear in the log if GClh is activated. #1688, dass ich auch dieses Mal nicht beseitigen konnte.

@capoaira
Copy link
Collaborator

Nach dem Einfügen eines Log Templates muss man noch eine Eingabe machen, damit der eingefügte Text tatsächlich vorhanden ist. Das ist das Problem aus Issue [New log form] Smiley inserted on the log page don't appear in the log if GClh is activated. #1688, dass ich auch dieses Mal nicht beseitigen konnte.

Das ist der selbe Grund, weshalb ich auch bei der Signatur noch nicht fertig bin.
Der Rest davon müsste in meiner PR funktionieren.

Gib einfach Bescheid wenn du keine Zeit hast.

Ich werde mich morgen dran setzen

@2Abendsegler
Copy link
Owner Author

Das ist der selbe Grund, weshalb ich auch bei der Signatur noch nicht fertig bin. Der Rest davon müsste in meiner PR funktionieren.

Gib einfach Bescheid wenn du keine Zeit hast.

Ich werde mich morgen dran setzen

Jetzt habe ich für die Anpassungen zur Signatur schon einen PR gemacht. 🙄😃

Wenn du noch etwas entwickeln möchtest, dann vielleicht den F2 im #2437. 😊

Und es steht ja auch einiges zum Testen an. Ich glaube damit hast du gut zu tun.
Ich kann dir auch noch meine Anpassungen zu den Notifications #2410 geben. 😁 Ich habe vor, die als Beta oder so, mit in die nächste Version zu bringen mit dem Hinweis, dass die noch nicht umfänglich getestet sind.

Copy link
Collaborator

@capoaira capoaira left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sieht gut aus 👍

Theoretisch könnten wir mit zumindest die Größe des Logfieds anpassen, indem wir ein input Event triggern (siehe mein PR zur Signatur #2441). Allerdings hat der Cacher dann vielleicht das Gefühl, dass alles übernommen wurde, was ja leider nicht der Fall ist. Also sollten wir das vielleicht lieber nicht tun.
Ich merge das hier erstmal.

@capoaira capoaira merged commit 3219332 into collector Nov 18, 2023
@capoaira capoaira added status: completed enhancement, improvement implemented and removed status: wait for merge labels Nov 18, 2023
@2Abendsegler
Copy link
Owner Author

Theoretisch könnten wir mit zumindest die Größe des Logfieds anpassen, indem wir ein input Event triggern (siehe mein PR zur Signatur #2441).

Das werde ich mir noch anschauen.

@2Abendsegler 2Abendsegler deleted the redesign-Log-Templates branch November 20, 2023 22:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status: completed enhancement, improvement implemented
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants